Feature/#102 refactor createshortgame #115

rkdmf1026 · 2023-10-05T07:35:09Z

📌 관련 이슈

closed [한판승부 생성] 한판승부 생성 변경사항 수정 #102

📷 screenshot

default.mp4

📝 Work Desciption

한판승부, 미션 카테고리 설명 뷰 구조 변경(Activity to Fragment)
한판승부 뷰 컴포넌트 위치 변경
한판승부 생성 기능 이전

📚 Reference 혹은 궁금한 사항들

액티비티뷰모델을 사용해서 하나의 뷰모델을 두 프래그먼트가 공유하는 식으로 구조를 변경했습니다.

taeheeL

+484 -481 가 좀 슬프네요.. 아빠 최고 ㅎ
xml 하시느라 고생이 많으십니다.. 리뷰 한번만 봐주세요!

taeheeL · 2023-10-05T10:21:08Z

app/src/main/java/sopt/uni/data/entity/shortgame/MissionContent.kt

-    @SerialName("image")
-    val image: String,
+    @SerialName("recommendTime")
+    val recommendTime: String?,


추천 시간이 null일 수도 잇나요..? 허걱

일단 서버가 내려주는게 nullable하게 내려줍니다...ㅜㅜ

taeheeL · 2023-10-05T10:29:32Z

app/src/main/java/sopt/uni/presentation/shortgame/createshortgame/CreateShortGameActivity.kt

+        if (fragment.isAdded) {
+            transaction.show(fragment)
+        } else {
+            transaction.add(R.id.fcv_create_short_game, fragment, tag)


replace가 아닌 hide & add 방식을 사용한 이유가 있을까요? 로직 설명해주시면 너무너무 감사하겠습니다 :)

replace와 add 차이를 보시면 replace는 이전 프래그먼트를 삭제하고 새로운 프래그먼트를 추가하는 것과 달리 add의 경우에는 프래그먼트를 삭제하지않습니다. 한판승부생성에서 미션카테고리뷰로 자주 이동을 해야할거라고 생각해서 굳이 매번 프래그먼트를 삭제할 필요없다고 생각해서 이렇게 구현했습니다!

저희 상황에서는 add로 show/hide를 하는 게 맞겠군요..! 상태를 보존해야 되기도 하니까
역시 아부지가 최고야 ㅎㅎ 감삼둥

taeheeL · 2023-10-05T10:42:44Z

...main/java/sopt/uni/presentation/shortgame/missiondetailcreate/MissionDetailCreateFragment.kt

+    private fun goToShortGame() {
+        val activity = activity as CreateShortGameActivity
+        activity.changeFragment(getString(R.string.label_short_game))
+    }


이렇게 하면 액티비티와 프래그먼트간의 강한 의존성이 생길 것 같은데, 다른 State나 Activity내에서 분기처리를 통해 변경시키는 건 어떨까요?

이 부분이 가장 고민했던 부분이긴한데, 메모장 기능 구현하고 시간 생기면 개선하도록 하겠습니다.

viewmodel에 state를 두고 액티비티에서 관찰해서 전환하는 식으로 해보겠습니다.

아빠 최고!

rkdmf1026 added 5 commits October 5, 2023 03:47

#102 / 미션상세 Activity to Fragment

7305c43

#102 / 한판승부생성 Activity to Fragment

56d2ee8

#102 / 미션카테고리 어댑터 수정

11121ab

#102 / String resource 수정

40fba79

#102 / API 변경에 따른 한판승부 관련 데이터 클래스 수정

4e2ce03

rkdmf1026 added feat 구현 및 개선 refactoring 리팩토링 UI UI 작업 동기 🐐 유니의 정신적 지주 담당 labels Oct 5, 2023

rkdmf1026 requested a review from a team October 5, 2023 07:35

rkdmf1026 self-assigned this Oct 5, 2023

taeheeL approved these changes Oct 5, 2023

View reviewed changes

taeheeL force-pushed the feature/#102-refactor-createshortgame branch from 8070a0b to 4e2ce03 Compare October 8, 2023 13:19

rkdmf1026 merged commit 8b86ed8 into develop Oct 20, 2023
1 check passed

rkdmf1026 deleted the feature/#102-refactor-createshortgame branch October 20, 2023 03:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/#102 refactor createshortgame #115

Feature/#102 refactor createshortgame #115

rkdmf1026 commented Oct 5, 2023

taeheeL left a comment

taeheeL Oct 5, 2023

rkdmf1026 Oct 8, 2023

taeheeL Oct 5, 2023

rkdmf1026 Oct 8, 2023

taeheeL Oct 11, 2023

taeheeL Oct 5, 2023

rkdmf1026 Oct 8, 2023

rkdmf1026 Oct 8, 2023

taeheeL Oct 11, 2023

Feature/#102 refactor createshortgame #115

Feature/#102 refactor createshortgame #115

Conversation

rkdmf1026 commented Oct 5, 2023

📌 관련 이슈

📷 screenshot

📝 Work Desciption

📚 Reference 혹은 궁금한 사항들

taeheeL left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment